Skip to content

Conversation

@nrc
Copy link
Member

@nrc nrc commented Nov 2, 2017

r? @eddyb

Nick Cameron added 4 commits November 2, 2017 08:12
This special cases the function type sugar in paths and deals with traits bounds as just the path parts. That required refactoring the path collector to distinguish between variable decls and references in patterns, basically just to please the borrow checker.

cc rust-dev-tools/rls-analysis#37
@eddyb
Copy link
Member

eddyb commented Nov 2, 2017

@bors r+

@bors
Copy link
Collaborator

bors commented Nov 2, 2017

📌 Commit 5d3be12 has been approved by eddyb

@kennytm kennytm added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Nov 2, 2017
@bors
Copy link
Collaborator

bors commented Nov 3, 2017

⌛ Testing commit 5d3be12 with merge 0030cab94cd7ab2ccf696eda3b37720c50d157dd...

@bors
Copy link
Collaborator

bors commented Nov 3, 2017

💔 Test failed - status-travis

@nrc
Copy link
Member Author

nrc commented Nov 3, 2017

@bors: retry

@bors
Copy link
Collaborator

bors commented Nov 3, 2017

⌛ Testing commit 5d3be12 with merge 8861eccb08cfe49231d436bd174c1ab433a249fe...

@bors
Copy link
Collaborator

bors commented Nov 3, 2017

💔 Test failed - status-travis

@kennytm
Copy link
Member

kennytm commented Nov 3, 2017

@bors retry

@bors
Copy link
Collaborator

bors commented Nov 4, 2017

⌛ Testing commit 5d3be12 with merge 98e4b68...

bors added a commit that referenced this pull request Nov 4, 2017
Fix a bunch of minor save-analysis bugs

r? @eddyb
@bors
Copy link
Collaborator

bors commented Nov 4, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: eddyb
Pushing 98e4b68 to master...

@bors bors merged commit 5d3be12 into rust-lang:master Nov 4, 2017
nrc pushed a commit to nrc/rust that referenced this pull request Nov 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants